Skip to content

chore: respect the new folder structure inside of Unity#2667

Merged
bitsandfoxes merged 6 commits into
mainfrom
chore/new-unity-structure
May 12, 2026
Merged

chore: respect the new folder structure inside of Unity#2667
bitsandfoxes merged 6 commits into
mainfrom
chore/new-unity-structure

Conversation

@bitsandfoxes
Copy link
Copy Markdown
Contributor

@bitsandfoxes bitsandfoxes commented May 6, 2026 •
edited
Loading

The layout inside of the Unity installation changed and we need to update the path resolution for the dependencies the SDK has at build time. Since mucking around inside the .targets is a pain I extracted this into a single script that gets invoked by MSBuild.

Added the new paths to the .targets.
Old layout (Unity ≤ 6000.1.x, macOS):

  /Applications/Unity/Hub/Editor/<version>/Unity.app/
  └── Contents/
      ├── MacOS/
      │   └── Unity
      ├── Managed/
      │   ├── UnityEngine.dll
      │   ├── UnityEditor.dll
      │   └── UnityEngine/
      │       └── UnityEngine.*.dll (modules)
      └── Resources/
          └── PackageManager/...

New layout (Unity 6000.3.x, macOS):

  /Applications/Unity/Hub/Editor/<version>/Unity.app/
  └── Contents/
      ├── MacOS/
      │   └── Unity
      └── Resources/
          ├── PackageManager/...
          └── Scripting/
              └── Managed/
                  ├── UnityEngine.dll
                  ├── UnityEditor.dll
                  └── UnityEngine/
                      └── UnityEngine.*.dll (modules)

@bitsandfoxes bitsandfoxes requested a review from a team May 7, 2026 09:39
cursor[bot]
cursor Bot reviewed May 7, 2026
View reviewed changes
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit e351381. Configure here.

Comment thread Directory.Build.targets Outdated
sentry[bot]
sentry Bot reviewed May 8, 2026
View reviewed changes
Comment thread scripts/find-unity.ps1 Outdated
@bitsandfoxes bitsandfoxes merged commit a081bd5 into main May 12, 2026
37 checks passed
@bitsandfoxes bitsandfoxes deleted the chore/new-unity-structure branch May 12, 2026 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sentry sentry[bot] sentry[bot] left review comments

@cursor cursor[bot] cursor[bot] left review comments

@JoshuaMoelans JoshuaMoelans JoshuaMoelans approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bitsandfoxes @JoshuaMoelans